-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
store/tikv: remove use of fastrand #22543
Conversation
Signed-off-by: disksing <i@disksing.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it affect the performance? Rest LGTM
@@ -213,7 +212,7 @@ func newTikvStore(uuid string, pdClient pd.Client, spkv SafePointKV, client Clie | |||
safePoint: 0, | |||
spTime: time.Now(), | |||
closed: make(chan struct{}), | |||
replicaReadSeed: fastrand.Uint32(), | |||
replicaReadSeed: rand.Uint32(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it affect the performance?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
forget it, it is not used so frequently
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@disksing merge failed. |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@disksing merge failed. |
/merge |
3 similar comments
/merge |
/merge |
/merge |
/run-all-tests |
@disksing merge failed. |
/run-all-tests |
Signed-off-by: disksing i@disksing.com
What problem does this PR solve?
Resolve dependency on
fastrand
.Part of #22513
What is changed and how it works?
What's Changed:
Use
math/rand
instead.Check List
Tests
Release note